Skip to content

ci(sanitization): install pytest so the gate actually runs#5

Merged
juniperbevensee merged 1 commit into
mainfrom
dev/juniperbevensee/ci-install-pytest
Jun 17, 2026
Merged

ci(sanitization): install pytest so the gate actually runs#5
juniperbevensee merged 1 commit into
mainfrom
dev/juniperbevensee/ci-install-pytest

Conversation

@juniperbevensee

Copy link
Copy Markdown
Collaborator

The sanitization workflow's self-test step runs python -m pytest but the install step only installed anthropic — so the gate failed with "No module named pytest" on every PR (4/4 runs since the workflow was created). The commons gate has therefore never actually enforced in CI; prior "clean" results were local runs only.

One-line fix: add pytest to the install line. This PR's own sanitize run should be the first green one.

🤖 Generated with Claude Code

The self-tests step ran `python -m pytest` but the install step only
installed `anthropic`, so the gate failed with "No module named pytest"
on every PR (4/4 runs, since inception). Add pytest to the install line
so the commons gate actually enforces in CI.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@juniperbevensee
juniperbevensee merged commit 39e4559 into main Jun 17, 2026
1 check passed
@juniperbevensee
juniperbevensee deleted the dev/juniperbevensee/ci-install-pytest branch June 17, 2026 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant